Skip to content

Conversation

@odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented Aug 22, 2025

What do these changes do?

This PR extends the Functions Browser with the following functionalities:

  • Permissions: Show to the user the same way the other access rights are presented
  • Searchable: Extended the search bar to support searching Functions

Next Step:

  • Allow sorting functions

Bonus:

  • Redesigned the Extended Search Bar to better handle the (longer) list of searchable contexts
  • Fixed: finally found the reason why sometimes the arrow in the drop down doesn't show up

Permissions:
FunctinonPermissions

Search:
FunctionSearch

Related issue/s

How to test

Dev-ops

@odeimaiz odeimaiz self-assigned this Aug 22, 2025
@odeimaiz odeimaiz added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Aug 22, 2025
@odeimaiz odeimaiz added this to the Voyager milestone Aug 22, 2025
@mergify
Copy link
Contributor

mergify bot commented Aug 22, 2025

🧪 CI Insights

Here's what we observed from your CI run for fe75253.

🟢 All jobs passed!

But CI Insights is watching 👀

@odeimaiz odeimaiz marked this pull request as ready for review August 25, 2025 14:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds permissions functionality and search capabilities for Functions in the dashboard. It implements a complete permissions system for Functions similar to other resources, enabling users to share Functions with specific access rights and search through them.

  • Enables Functions to be shared with read/write permissions like other resources
  • Implements search functionality for Functions
  • Adds UI components and backend integration for Function permissions management

Reviewed Changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
osparc/ui/list/CollaboratorListItem.js Adds Function role support and improves permission checking logic
osparc/theme/Appearance.js Adds styling for selectbox arrow component
osparc/store/Store.js Adds search Functions context to store
osparc/store/Groups.js Removes unused curateOrderBy method
osparc/store/Functions.js Adds search, permissions management, and collaboration methods
osparc/share/NewCollaboratorsManager.js Syncs arrow appearance for access rights dropdown
osparc/share/Collaborators*.js Adds default access rights comments and Function collaborator support
osparc/share/CollaboratorsFunction.js New class implementing Function-specific collaboration functionality
osparc/desktop/credits/CreditsIndicatorButton.js Removes unused offset parameter
osparc/data/model/Function.js Adds canIWrite permission checking method
osparc/data/Roles.js Defines Function-specific roles and permissions
osparc/data/Resources.js Adds Function permission API endpoints
osparc/dashboard/*.js Implements Function search UI and context handling

@odeimaiz odeimaiz added the 🤖-automerge marks PR as ready to be merged for Mergify label Aug 25, 2025
@odeimaiz
Copy link
Member Author

odeimaiz commented Aug 25, 2025

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Aug 25, 2025

queue

🛑 Configuration not compatible with a branch protection setting

The branch protection setting Require branches to be up to date before merging is not compatible with max_parallel_checks>1, queue_conditions != merge_conditions and must be unset.

@odeimaiz odeimaiz changed the title ✨ [Frontend] Functions permissions ✨ [Frontend] Functions Browser: Permissions and Search Aug 25, 2025
Copy link
Contributor

@wvangeit wvangeit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @odeimaiz !

@sonarqubecloud
Copy link

@odeimaiz odeimaiz merged commit 2755211 into ITISFoundation:master Aug 25, 2025
58 checks passed
@matusdrobuliak66 matusdrobuliak66 mentioned this pull request Sep 2, 2025
61 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🤖-automerge marks PR as ready to be merged for Mergify a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Frontend] Sort and Filter functions [Frontend] Permissions on Function

4 participants